New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
e2e network policy enclose IPv6 destinations #96856
Conversation
We moved to DNS destination to IP:Port, so we need to be sure that IPv6 addresses are enclosed in square brackets with that format.
/sig network |
Labels: map[string]string{ | ||
"special-label": f.UniqueName, | ||
}}, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is my linter things 🤷
/milestone v1.20
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, spiffxp The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/priority important-soon |
Hey @spiffxp and @aojea, we're about to cut v1.20.0-rc.0 which also creates the release-1.20 branch. From that point on we have to cherry-pick all changes which should be included in v1.20.0. From my perspective this PR is fine to be merged, but I'd like to give @jeremyrickard a head nod. |
thanks @saschagrunert , @jeremyrickard I appreciate if we can have it in 1.20, as you can see, network policies tests with ipv6 doesn't work 🤷 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/hold cancel |
/retest This sounds familiar :) |
We moved from testing against a DNS destination to use IP:Port,
so we need to be sure that IPv6 addresses are enclosed in square
brackets with that format.
/kind bug
The test now fails with:
because of